Skip to content

fix: warnet setup bump pinned kubectl to v1.35.1#800

Merged
pinheadmz merged 2 commits intobitcoin-dev-project:mainfrom
b-l-u-e:fix/kubectl-version-skew-warnet-setup
Apr 23, 2026
Merged

fix: warnet setup bump pinned kubectl to v1.35.1#800
pinheadmz merged 2 commits intobitcoin-dev-project:mainfrom
b-l-u-e:fix/kubectl-version-skew-warnet-setup

Conversation

@b-l-u-e
Copy link
Copy Markdown
Contributor

@b-l-u-e b-l-u-e commented Mar 25, 2026 •
edited
Loading

Minikube is provisioning Kubernetes v1.35.1....
Digital ocean recommends Kubernetes 1.33, so with kubectl v1.35.1 the client is 2 minor versions ahead
this should fix #747

 minikube start
😄  minikube v1.38.1 on Ubuntu 24.04
✨  Automatically selected the docker driver
❗  Starting v1.39.0, minikube will default to "containerd" container runtime. See #21973 for more info.
📌  Using Docker driver with root privileges
👍  Starting "minikube" primary control-plane node in "minikube" cluster
🚜  Pulling base image v0.0.50 ...
💾  Downloading Kubernetes v1.35.1 preload ...
    > preloaded-images-k8s-v18-v1...:  272.45 MiB / 272.45 MiB  100.00% 1.22 Mi
    > gcr.io/k8s-minikube/kicbase...:  519.58 MiB / 519.58 MiB  100.00% 846.93 
🔥  Creating docker container (CPUs=2, Memory=3800MB) ...
🐳  Preparing Kubernetes v1.35.1 on Docker 29.2.1 ...
🔗  Configuring bridge CNI (Container Networking Interface) ...
🔎  Verifying Kubernetes components...
    ▪ Using image gcr.io/k8s-minikube/storage-provisioner:v5
🌟  Enabled addons: storage-provisioner, default-storageclass

❗  /home/user/projects/warnet/warnet/.venv/bin/kubectl is version 1.31.1, which may have incompatibilities with Kubernetes 1.35.1.
    ▪ Want kubectl v1.35.1? Try 'minikube kubectl -- get pods -A'
🏄  Done! kubectl is now configured to use "minikube" cluster and "default" namespace by default

@b-l-u-e b-l-u-e force-pushed the fix/kubectl-version-skew-warnet-setup branch from cbf913a to 031e245 Compare March 25, 2026 13:59
@pinheadmz pinheadmz force-pushed the fix/kubectl-version-skew-warnet-setup branch from 031e245 to d14bfca Compare April 21, 2026 14:21
pinheadmz
pinheadmz reviewed Apr 21, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@pinheadmz pinheadmz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I double checked the shasums, and cleaned up some nits

Comment thread .github/workflows/test.yml Outdated
push:
branches:
- '**'
- "**"
Copy link
Copy Markdown
Contributor

@pinheadmz pinheadmz Apr 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need to lint code you are not changing like this

@b-l-u-e @pinheadmz
warnet setup now installs kubectl v1.35.1 (with updated checksums) an…
565692b 
…d CI runs minikube/kubectl v1.35.1 to avoid kubectl version-skew issues.
@pinheadmz pinheadmz force-pushed the fix/kubectl-version-skew-warnet-setup branch from d14bfca to 565692b Compare April 22, 2026 13:20
@pinheadmz
Copy link
Copy Markdown
Contributor

pinheadmz commented Apr 22, 2026

rebased on main

@pinheadmz
Copy link
Copy Markdown
Contributor

pinheadmz commented Apr 22, 2026

ok the updated version of minkube is more strict about authorization policy, so i had to change the rbac.yaml for the scenario commander as well, hopefully this passes ci now!

@pinheadmz pinheadmz merged commit aedca69 into bitcoin-dev-project:main Apr 23, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pinheadmz pinheadmz pinheadmz left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

update helm and kubectl "blessed" versions

2 participants

@b-l-u-e @pinheadmz